Skip to content

Fix(engine): Fix scheduled nightly clippy#160

Merged
joshuajbouw merged 2 commits into
developfrom
fix-more-needless-borrows
Jun 25, 2021
Merged

Fix(engine): Fix scheduled nightly clippy#160
joshuajbouw merged 2 commits into
developfrom
fix-more-needless-borrows

Conversation

@birchmd
Copy link
Copy Markdown
Member

@birchmd birchmd commented Jun 24, 2021

The scheduled run of nightly clippy was broken. If it would have run it would have also found two more needless borrows, so those are fixed here too.

@birchmd birchmd requested review from artob and joshuajbouw June 24, 2021 13:51
Copy link
Copy Markdown
Contributor

@sept-en sept-en left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Copy Markdown
Contributor

@joshuajbouw joshuajbouw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love these new clippy errors. Good to go.

@joshuajbouw joshuajbouw merged commit 41b1a7d into develop Jun 25, 2021
@joshuajbouw joshuajbouw deleted the fix-more-needless-borrows branch June 25, 2021 14:06
artob added a commit that referenced this pull request Jul 8, 2021
* Fix scheduled nightly clippy. (#160)
* Always compile tests with the `mainnet` feature. (#162)
* Move eth-connector tests to under `src/tests`. (#164)
* EIP-2718: Support typed transaction envelopes. (#165)
* Remove deprecated EIP-712 prover implementation. (#168)
* ERC-20: Allow admin to change token metadata. (#171)

Co-authored-by: Evgeny Ukhanov <evgeny@aurora.dev>
Co-authored-by: Kirill <kirill@aurora.dev>
Co-authored-by: Michael Birch <michael@aurora.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants